Skip to content

PoC feature_in_id for single module repositories#604

Open
MaximilianSoerenPollak wants to merge 4 commits into
eclipse-score:mainfrom
MaximilianSoerenPollak:MSP_poc_feature_id
Open

PoC feature_in_id for single module repositories#604
MaximilianSoerenPollak wants to merge 4 commits into
eclipse-score:mainfrom
MaximilianSoerenPollak:MSP_poc_feature_id

Conversation

@MaximilianSoerenPollak

@MaximilianSoerenPollak MaximilianSoerenPollak commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

📌 Description

This is a suggestion/PoC on how we could allow for the feature_in_id check to be kept but adapt it so that single feature /module repositories also pass this check.
This is done here by adding an additional config parameter to the conf.py that defines strings that are also allowed in the feature_part of the id.
Now only if NONE of the 3 things (docpath, abbreviation or configured param) are found an error is thrown.

Fixes #596

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as draft June 16, 2026 13:57
@MaximilianSoerenPollak MaximilianSoerenPollak changed the title Msp poc feature PoC feature_in_id for single module repositories Jun 16, 2026
@github-actions

github-actions Bot commented Jun 16, 2026 •
edited
Loading

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: c6e502f7-36d1-464d-8325-d53d08175762
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 10.204s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@MaximilianSoerenPollak MaximilianSoerenPollak marked this pull request as ready for review June 18, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexanderLanin AlexanderLanin Awaiting requested review from AlexanderLanin AlexanderLanin is a code owner

@dcalavrezo-qorix dcalavrezo-qorix Awaiting requested review from dcalavrezo-qorix dcalavrezo-qorix is a code owner

@nradakovic nradakovic Awaiting requested review from nradakovic nradakovic is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

INF - Infrastructure Community
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Create PoC to showcase 'feature in id' functionality

1 participant

@MaximilianSoerenPollak